Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All package update #107

Merged
merged 6 commits into from
Mar 16, 2024
Merged

All package update #107

merged 6 commits into from
Mar 16, 2024

Conversation

rv2931
Copy link
Collaborator

@rv2931 rv2931 commented Mar 13, 2024

Attempt to upgrade all packages

Evolutions:

  • BaseSetting in bloom.config has moved to pydantic-settings package. This package a built-in package that suppply all mecanism that was put in place in bloom.config and better so migrated to pydantic-settings

@rv2931
Copy link
Collaborator Author

rv2931 commented Mar 14, 2024

Après migration de bloom.settings pour du pydantic-settings ça m'a l'air de fonctionner par contre il va falloir qu'un certain nombre de testeurs passent pour valider l'upgrade

@rv2931
Copy link
Collaborator Author

rv2931 commented Mar 14, 2024

Pour le moment ne fonctionne pas. Cf #108

@rv2931
Copy link
Collaborator Author

rv2931 commented Mar 15, 2024

Cette PR semble fonctionner
Testé en Docker compose et juste sur la partie TrawlWatcher, visualisation du mmsi 261084090 , affichage du voyage_id 1 (toujours le bug sur le dernier voyage_id) #100

@njouanin
Copy link
Collaborator

njouanin commented Mar 15, 2024

OK pour moi sur le test de lancement de app.py (qui lance le scraping depuis Spire):
le script se lance sans erreur (récupération de la config OK donc) mais plante car il ne trouve pas variable d'environnement contenant le token spire. C'est normal.

@rv2931
Copy link
Collaborator Author

rv2931 commented Mar 16, 2024

Hello
Normalement (faut que je pense à vérifier) la token spire est maintenant prise a partir de bloom.config et non l'environnement direct. C'est important car le mécanisme de chargement du paramétrage utilise les fichiers et l'env donc faudra veiller à ce que tout prenne ses paramètre a partir de bloom.config
Sinon pour cette PR je sais que t'es plutôt pour la merger. Y a pas eu trop de répondant sur d'autres testeurs pour le moment
On se dit que je la merge aujourd'hui fin de journée
Idéalement refait un tour côté add ports db (2) pour la revalider et je meegerai les deux

@rv2931 rv2931 marked this pull request as ready for review March 16, 2024 08:55
@rv2931 rv2931 merged commit 46b2eff into main Mar 16, 2024
0 of 2 checks passed
@rv2931 rv2931 deleted the all_package_update branch March 16, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants